Walkthrough

InsertTextAtCursor now computes replacement boundaries using TrueSelectionStartWithoutLineBreaks and TrueSelectionEndWithoutLineBreaks when a selection exists, ensuring correct start/end positions regardless of selection direction. No other logic in the method changed.

Changes

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant U as User
  participant E as EditableTextBase
  participant S as Selection State
  participant D as Document Buffer

  U->>E: InsertTextAtCursor(text)
  E->>S: Check selection range
  alt Selection exists
    S-->>E: TrueSelectionStart/End (no line breaks)
    note over E: Use TrueSelection* to compute replacement
    E->>D: Replace [start:end] with text
  else No selection
    E->>D: Insert text at caret position
  end
  E->>S: Update caret and selection
  E-->>U: Return

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I nibble on code like clover at dawn,
Flip-flopped selections now tidy and drawn.
True starts, true ends, no lines astray—
Cursor hops lightly, on its way.
In fields of text, I bop and weave,
A happy hare with tricks up sleeve. 🐇✨

Pre-merge checks and finishing touches

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}